Skip to content

104 folder upload download#315

Open
CowMan999 wants to merge 6 commits intomainfrom
104-folder-upload-download
Open

104 folder upload download#315
CowMan999 wants to merge 6 commits intomainfrom
104-folder-upload-download

Conversation

@CowMan999
Copy link

Solves #104 by allowing folder uploads via drag&drop and the file prompt

@CowMan999 CowMan999 linked an issue Nov 30, 2025 that may be closed by this pull request
Copy link
Collaborator

@gr812b gr812b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll take a closer look at this later today, just wondering - I don't see functionality for folder download as well, could you add this too!

export const UploadModal = ({ isOpen, onClose }: UploadModalProps) => {
const [files, setFiles] = useState<File[]>([]);
const { refetch } = useFiles();
const [allowFolder, setAllowFolder] = useState(false);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this toggle needed here? I like the fact you can disable it, for the presetFilesModal, but for upload I think it should be okay to always allow folders as well as normal files too

Seems you cannot have folder and file enabled as the same selection mechanism, choice between two buttons or a toggle
@gr812b
Copy link
Collaborator

gr812b commented Jan 16, 2026

Is this toggle needed here? I like the fact you can disable it, for the presetFilesModal, but for upload I think it should be okay to always allow folders as well as normal files too

Turns out you had it for good reason! I pushed a change copying similar UIs in just having two buttons instead of a toggle mode, upload form should be good to go, I left a bit of cleanup in the upload modal, and also the download modal could be implemented too. Maybe just letting someone shift + click to select a whole folder

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Folder upload / download

2 participants